Skip to content

[CHORE] Promote @MikeGoldsmith to Approver - #50950

Merged
MovieStoreGuy merged 1 commit into
open-telemetry:mainfrom
MovieStoreGuy:msg/promote-mike-goldsmith
Sep 14, 2026
Merged

[CHORE] Promote @MikeGoldsmith to Approver#50950
MovieStoreGuy merged 1 commit into
open-telemetry:mainfrom
MovieStoreGuy:msg/promote-mike-goldsmith

Conversation

@MovieStoreGuy

@MovieStoreGuy MovieStoreGuy commented Sep 13, 2026

Copy link
Copy Markdown
Contributor

Description

Promoting @MikeGoldsmith to approver.
@MikeGoldsmith is code owner for various components, and I have worked closely on with the schema processor.
Having Mike as part of the approver group would bring a well balanced perspective in how we do work.

Authorship

  • I, a human, wrote this pull request description myself.

@MovieStoreGuy
MovieStoreGuy requested a review from a team as a code owner September 13, 2026 23:58
@MovieStoreGuy

MovieStoreGuy commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

cc: @open-telemetry/collector-contrib-contributors @open-telemetry/collector-maintainers

singhvibhanshu
singhvibhanshu approved these changes Sep 14, 2026
View reviewed changes

@singhvibhanshu singhvibhanshu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@YElayyat

YElayyat commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

👏

himanshu130700
himanshu130700 approved these changes Sep 14, 2026
View reviewed changes

@himanshu130700 himanshu130700 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@opentelemetry-pr-dashboard

opentelemetry-pr-dashboard Bot commented Sep 14, 2026

Copy link
Copy Markdown

Pull request dashboard status

Waiting on maintainers · refreshed 2026-09-14 05:28 UTC

Merge when ready.

Status above doesn't look right?
  • Just replied or pushed? Anything around or after the refresh time above may not be picked up yet — give it a few minutes.
  • Anything look wrong? Report it with what you expected; it helps us improve the dashboard.

@opentelemetry-pr-dashboard opentelemetry-pr-dashboard Bot mentioned this pull request Sep 14, 2026
Open
paulojmdias
paulojmdias approved these changes Sep 14, 2026
View reviewed changes

@paulojmdias paulojmdias left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🎉

@jpkrohling

jpkrohling commented Sep 14, 2026

Copy link
Copy Markdown
Member

I know my support can't be official as I don't hold any positions at this SIG, but as an outsider and former contributor to the SIG, I fully support @MikeGoldsmith's nomination

@MovieStoreGuy

MovieStoreGuy commented Sep 14, 2026

Copy link
Copy Markdown
Contributor Author

We have agreement with the majority, please welcome @MikeGoldsmith !

@MovieStoreGuy
MovieStoreGuy merged commit 7378d30 into open-telemetry:main Sep 14, 2026
85 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TylerHelmuth TylerHelmuth TylerHelmuth approved these changes
@braydonk braydonk braydonk approved these changes
@paulojmdias paulojmdias paulojmdias approved these changes
@codeboten codeboten codeboten approved these changes
@ChrsMark ChrsMark ChrsMark approved these changes
@evan-bradley evan-bradley evan-bradley approved these changes
@songy23 songy23 songy23 approved these changes
@dmitryax dmitryax dmitryax approved these changes
@mx-psi mx-psi mx-psi approved these changes
@iblancasa iblancasa iblancasa approved these changes
@jmacd jmacd jmacd approved these changes
+11 more reviewers
@himanshu130700 himanshu130700 himanshu130700 approved these changes
@jkoronaAtCisco jkoronaAtCisco jkoronaAtCisco approved these changes
@harshitt13 harshitt13 harshitt13 approved these changes
@singhvibhanshu singhvibhanshu singhvibhanshu approved these changes
@osullivandonal osullivandonal osullivandonal approved these changes
@emreyalvac emreyalvac emreyalvac approved these changes
@dao-jun dao-jun dao-jun approved these changes
@NassimBtk NassimBtk NassimBtk approved these changes
@dyl10s dyl10s dyl10s approved these changes
@cparkins cparkins cparkins approved these changes
@mwear mwear mwear approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.